@rubytech/create-realagent 1.0.649 → 1.0.651

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rubytech/create-realagent",
-  "version": "1.0.649",
+  "version": "1.0.651",
   "description": "Install Real Agent — Built for agents. By agents.",
   "bin": {
     "create-realagent": "./dist/index.js"

package/payload/platform/plugins/cloudflare/scripts/list-cf-domains.ts

@@ -25,12 +25,18 @@
 //            "selector drift" from "empty account" — both shapes arrive via
 //            stdout).
 
+import { appendFileSync } from "node:fs";
 import { writeFile } from "node:fs/promises";
 import { resolve } from "node:path";
 import { homedir } from "node:os";
 
-const CDP_HOST = "127.0.0.1";
-const CDP_PORT = 9222;
+// CDP host/port default to the VNC Chromium's localhost bind; env overrides
+// exist solely so the vitest regression under platform/ui/__tests__ can force
+// a deterministic ECONNREFUSED (point at 127.0.0.1:1 — RFC 6335 reserved,
+// guaranteed-refused on every platform) without depending on whether a dev
+// machine happens to have VNC Chromium running on 9222.
+const CDP_HOST = process.env.LIST_CF_DOMAINS_CDP_HOST ?? "127.0.0.1";
+const CDP_PORT = Number(process.env.LIST_CF_DOMAINS_CDP_PORT ?? 9222);
 
 // Phase budgets total 30s (enforced by the shell wrapper's `timeout`).
 // Individual steps get sub-budgets so an early stall does not eat the
@@ -50,11 +56,42 @@ type Reason =
   | "table-wait-timeout"
   | "runtime-error";
 
+// Sticky flag: once a stream-log write fails (EACCES, ENOENT, …), skip
+// subsequent file writes for the rest of this process. Repeated appends to an
+// unwritable path would spam stderr with identical `phase=stream-log-write-failed`
+// lines and crowd result.stderr that the route handler regex reads. Losing a
+// log line is strictly better than aborting the scrape — observability failure
+// must not mask the scrape's real signal.
+let streamLogWriteFailed = false;
+
 function logPhase(line: string): void {
-  // Written to stderr; shell wrapper tees to STREAM_LOG_PATH with timestamp.
-  // Keeping format parity with `_stream-log.sh phase_line` on the bash side
-  // means the tailer regex and grepping by `[list-cf-domains]` work uniformly.
+  // Stderr: preserved for direct-SSH invocation parity AND for runFormSpawn's
+  // in-memory `result.stderr`, which the route handler greps for `reason=<enum>`
+  // on non-zero exit. Removing this write would break the form's error-card path.
   process.stderr.write(`[list-cf-domains] ${line}\n`);
+
+  // Stream log: direct write by the TS helper (not via wrapper tee) so phase
+  // lines reach the per-conversation file the Task 592 tailer reads, even on
+  // exit 0 where runFormSpawn discards `result.stderr`. Format parity with
+  // `_stream-log.sh phase_line`: `[<ISO-ts>] [<scope>] <content>\n`, asserted
+  // by the vitest regression under platform/ui/__tests__. Sync append so the
+  // terminal `phase=script-exit code=0 count=N` lands before process exit.
+  const streamLogPath = process.env.STREAM_LOG_PATH;
+  if (!streamLogPath || streamLogWriteFailed) return;
+  try {
+    appendFileSync(
+      streamLogPath,
+      `[${new Date().toISOString()}] [list-cf-domains] ${line}\n`,
+    );
+  } catch (err) {
+    streamLogWriteFailed = true;
+    const detail = (err instanceof Error ? err.message : String(err))
+      .slice(0, 200)
+      .replace(/"/g, "'");
+    process.stderr.write(
+      `[list-cf-domains] phase=stream-log-write-failed detail="${detail}"\n`,
+    );
+  }
 }
 
 function die(reason: Reason, detail = ""): never {